Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MIRROR] Emagged PDAs won't log to wirecarp. #626

Merged
merged 1 commit into from
Nov 15, 2023
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25004
Original PR: tgstation/tgstation#79707

About The Pull Request

Currently, every time you open/close a program, that activity is logged for the RD to see. This includes Nukies opening their disk tracking app.
Now, emagged PDAs will no longer log activities to Wirecarp, but I did replace it with a log when the PDA is first emagged.
Basically, nukies are undetected, but Traitors can still be figured out.

Why It's Good For The Game

Closes tgstation/tgstation#77201

I thought this would be better as it makes more sense to stop logging when you're emagged, but as you are being emagged it will report the modification instead.

Changelog

🆑 JohnFulpWillard
fix: PDAs now log that they've been emagged, but will no longer log any further programs they open beyond that. This means Nukies don't sell themselves out by opening their disk tracking app.
/:cl:

* Emagged PDAs won't log to wirecarp. (#79707)

## About The Pull Request

Currently, every time you open/close a program, that activity is logged
for the RD to see. This includes Nukies opening their disk tracking app.
Now, emagged PDAs will no longer log activities to Wirecarp, but I did
replace it with a log when the PDA is first emagged.
Basically, nukies are undetected, but Traitors can still be figured out.

## Why It's Good For The Game

Closes tgstation/tgstation#77201

I thought this would be better as it makes more sense to stop logging
when you're emagged, but as you are being emagged it will report the
modification instead.

## Changelog

:cl:
fix: PDAs now log that they've been emagged, but will no longer log any
further programs they open beyond that. This means Nukies don't sell
themselves out by opening their disk tracking app.
/:cl:

* Emagged PDAs won't log to wirecarp.

---------

Co-authored-by: John Willard <[email protected]>
@Iajret Iajret merged commit 268c261 into master Nov 15, 2023
24 checks passed
@Iajret Iajret deleted the upstream-mirror-25004 branch November 15, 2023 09:12
AnywayFarus added a commit that referenced this pull request Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WireCarp can be used to detect nuclear PDAs when they open the software hub
3 participants